Skip to content

http: add absolute request-target support (fixes #193)#651

Merged
arnetheduck merged 5 commits into
masterfrom
conn-prov
Jun 15, 2026
Merged

http: add absolute request-target support (fixes #193)#651
arnetheduck merged 5 commits into
masterfrom
conn-prov

Conversation

@arnetheduck

Copy link
Copy Markdown
Member

Using an absolute request URI and a connection provider that connects to a different server than the http address, generic proxies can be implemented

  • add detail to http connection error
  • fix transport/stream leak when http client construction fails

@arnetheduck

Copy link
Copy Markdown
Member Author

this is an alternative to #618

Comment thread chronos/apps/http/httpclient.nim Outdated
Comment thread chronos/apps/http/httpserver.nim Outdated
Comment thread chronos/apps/http/httpserver.nim Outdated
Comment thread chronos/apps/http/httpclient.nim Outdated
Comment thread chronos/apps/http/httpclient.nim Outdated
arnetheduck and others added 4 commits June 1, 2026 18:50
Using an absolute request URI and a connection provider that connects to
a different server than the http address, generic proxies can be
implemented

* add detail to http connection error
* fix transport/stream leak when http client construction fails
Co-authored-by: Jacek Sieka <arnetheduck@gmail.com>
@arnetheduck arnetheduck merged commit b76a78f into master Jun 15, 2026
49 of 50 checks passed
@arnetheduck arnetheduck deleted the conn-prov branch June 15, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants